Skip to content

chore: update dep remove hard coded deps#59

Merged
runreal-warman merged 2 commits intomainfrom
update-deps
Apr 14, 2025
Merged

chore: update dep remove hard coded deps#59
runreal-warman merged 2 commits intomainfrom
update-deps

Conversation

@thibaultleouay
Copy link
Copy Markdown
Contributor

No description provided.

@thibaultleouay thibaultleouay requested a review from Copilot April 14, 2025 07:48
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot reviewed 7 out of 8 changed files in this pull request and generated 1 comment.

Files not reviewed (1)
  • deno.jsonc: Language not supported

import * as path from '@std/path'
import { $ } from '@david/dax'
import type { Command } from '@cliffy/command'
import type * as path from '@std/path'
Copy link

Copilot AI Apr 14, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TypeScript does not support type-only namespace imports. Remove the 'type' keyword or refactor the import to use named type-only imports if only types are needed.

Suggested change
import type * as path from '@std/path'
import * as path from '@std/path'
import type { PathLike, ParsedPath } from '@std/path'

Copilot uses AI. Check for mistakes.
@thibaultleouay thibaultleouay requested a review from Copilot April 14, 2025 08:07
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot reviewed 7 out of 8 changed files in this pull request and generated no comments.

Files not reviewed (1)
  • deno.jsonc: Language not supported
Comments suppressed due to low confidence (2)

src/commands/uasset/parse.ts:103

  • Ensure that the additional double space in the template literal is intentional, as it may affect the parsing logic if extra whitespace is not expected.
complexValue += `  ${lines[i].trim()}`

src/commands/script.ts:7

  • Verify that the local 'esbuild' dependency is correctly installed and available in the project as the removal of the URL-based import may lead to module resolution issues.
import * as esbuild from 'esbuild'

@runreal-warman runreal-warman merged commit ef98cb0 into main Apr 14, 2025
2 checks passed
@runreal-warman runreal-warman deleted the update-deps branch April 29, 2025 14:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants